Skip to content

Post-merge-review: Fix template-quotes: accept boolean root config#2696

Merged
NullVoxPopuli merged 3 commits intoember-cli:masterfrom
johanrd:day_fix/template-quotes
Apr 14, 2026
Merged

Post-merge-review: Fix template-quotes: accept boolean root config#2696
NullVoxPopuli merged 3 commits intoember-cli:masterfrom
johanrd:day_fix/template-quotes

Conversation

@johanrd
Copy link
Copy Markdown
Contributor

@johanrd johanrd commented Apr 13, 2026

Summary

  • Schema now accepts false as root config (matches upstream's [rule, false] disabled state)
  • create() short-circuits when config is false

Test plan

  • options: [false] with any quote style → no errors reported

johanrd added 3 commits April 13, 2026 14:43
Upstream accepts [rule, false] to disable the rule; port's schema
rejected it. Allow boolean root (false = disabled) in the schema and
short-circuit create() when disabled.
@johanrd johanrd marked this pull request as ready for review April 13, 2026 17:18
@johanrd johanrd force-pushed the day_fix/template-quotes branch from 851ebcc to 2f0dce9 Compare April 13, 2026 17:38
@johanrd johanrd changed the title Fix template-quotes: accept boolean root config Post-merge-review: Fix template-quotes: accept boolean root config Apr 13, 2026
@NullVoxPopuli NullVoxPopuli merged commit cda8bed into ember-cli:master Apr 14, 2026
19 of 20 checks passed
@github-actions github-actions bot mentioned this pull request Apr 14, 2026
@johanrd johanrd deleted the day_fix/template-quotes branch April 14, 2026 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants